icon theme: Make final

Nobody should subclass GtkIconTheme or GtkIconInfo.

textview: Use standard padding

Given that GtkSourceView exists, we will have
to keep GtkTextView subclassable until a replacement
appears.

viewport: Cleanup

viewport: Make final

No need to derive from it.

Trivial

surface: fix up docs

The docs for gdk_surface_invalidate_region were
talking about a no-longer-existing argument.

color plane: Don't allocate 0-size textures

This triggers critials, and does us no good.

Cosmetics

rendernodeparser: Parse Cairo nodes

We also print the script, but we don't parse it yet.

rendernode: Remove unused function

inspector: Make updates overlay not crash

... when a window gets hidden and later reshown.

The code now properly cleans up a window when it gets unmapped instead
of trying to retain previous updates information.

Revert "wayland: Keep some resources until destroy"

This reverts commit f2d598b9a19cd7151c592e9c74d5b0d0ba3c9e48.

widget-factory: Add some animation in a popover

broadway: add frame times to the profiler

fishbowl: Add a menubutton demo

This reveals some issues with the position tracking.

menu button: Add popup/popdown functions

These are needed, after all.

fishbowl: Fix the lock button

gl renderer: Make creating render targets easier

gl renderer: Remove some unused api form the GLDriver

fishbowl: Don't show ridiculous precision

Two decimals are more than enough for the frame rate.

Merge branch 'issue1901-gtk4' into 'master'

wayland: Use scale from core wl_output protocol

Closes #1901

See merge request GNOME/gtk!870

wayland: Use scale from core wl_output protocol

The “xdg-output” protocol provides clients with the outputs size and
position in compositor coordinates, and does not provide the output
scale which is already provided by the core “wl_output” protocol.

So when receiving the wl_output scale event, we should update the scale
regardless of “xdg-output” support, otherwise the scale will remain to
its default value of 1 and the surface will be scaled up by the
compositor to match the actual output scale, which causes blurry fonts
and widgets.

Fixes: https://gitlab.gnome.org/GNOME/gtk/issues/1901
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

gtk-demo: Reinstate flat menu buttons

widget-factory: Reinstate flat menu buttons

menu button: Add a relief property

This is used in various places.

testscrolledwindow: GtkMenuButton != GtkContainer

It's not anymore, so don't cast it to one... Use the new :label instead.

testsuite: Fix leak

Merge branch 'mkenums-once' into 'master'

Use g_once for all the mkenums template files

Closes #1617

See merge request GNOME/gtk!865

Use g_once for all the mkenums template files

Without this, initialization of enum/flags values from multithreaded
programs may not work correctly.

Fixes #1617.

Merge branch 'win32-enable-werror' into 'master'

Win32 enable werror

See merge request GNOME/gtk!861

Merge branch 'issue-302-master' into 'master'

docs: Fix the override for GdkAtom

Closes #302

See merge request GNOME/gtk!864

docs: Fix the override for GdkAtom

GdkAtom is a typedef to a pointer to an opaque structure. We need to
tell GTK-Doc how to override it, so that the documentation is accurate.

Fixes: #302

gl renderer: Render recording surface untransformed

Cairo can't handle the y flip apparently.

Fixes #1906

gl glyphcache: Don't choke on huge glyphs

We blindly assume everywhere that a single glyph will definitely fit on
one atlas, but that's not always the case.
For now, don't crash or produce GL errors.

widget: Don't always call get_display() when allocating

The GTK_DISPLAY_DEBUG_CHECK macro will cleverly only call the function
if any of the display debug flags are set, so in the common case it
won't even be executed.

win32: Fix meson names for defines

Those have been wrong forever, but nobody cares because they get to the
values we want anyway.

win32: Enable --werror in CI

win32: Remove outdated workaround

The function that was missing in 2016 does exist now.

Fixes gcc complaining about a wrong declaration.

tests: Use g_free(), not free()

Windows doesn't like that.

imcontextime: Add missing semicolon

I wonder how thoroughly this was tested... ;)

win32: Add missing enum value to switch statement

imcontextime: Remove unused variables

win32: Rename a shadowing variable

window: Remove unneeded variable

The variable would show up as a gcc warning in builds without X11
support.

win32: Actually pass y variable for y parameter

win32: Remove unused variables

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!860

testsuite: Remove serialize-deserialize test

The serialize-deserialize test is completely handled via the new
nodeparser tests.

So move those tests over and delete the old test binary.

testsuite: Add render tests for all the empty nodes

Test that rendering empty nodes succees. For a lot of nodes the
resulting rendering isn't clearly defined, in those cases we overdraw
those regions (sometimes the whole image) with black.

testuite: Improve compare-render binary

- Remove remains of g_test_*() functions
  We're not a glib test, we're a simple binary.
- Handle nonexistence of reference image properly
  Don't assert, but create the output image and the error out.

rendernode: Blur alpha channel for cairo blur node

The blur node still doesn't properly blur with a transform, but at least
it now doesn't turn semi-transparent stuff black.

testsuite: Add empty node tests

For every node type, add a test that creates it empty and ensure that
it prints the reference output.

rendernodeparser: Allow parsing strings

Instead of only allowing for glyph indexes, allow ASCII characters as
replacements. So this glyph sequence
  glyphs: 65 8, 66 8, 67 8
Can be replaced by
  glyphs: "ABC"
provided that the glyph for "A", "B" and "C" are 65, 66 and 67
respectively and their advance is exactly 8.
x offset and y offset must always be 0 and every glyph must start a
cluster.

rendernodeparser: Update to new rules

Update to the docs outlined in #1887.

In particular, the changes do:

1. Require no property, have a working default for everything
2. Be clear about what gets printed and how.

Tests ahve been adapted to still pass.

rgba: Add GDK_RGBA() macro

So far it's private, but it's a pretty cute way to use hex colors, so we
might conside making it public.

rendernode: Take a graphene_point_t for the offset

... instead of 2 floats.

testsuite: Redo node-parser

Base the rewrite on testsuite/css/parser/test-css-parser - we now
require the node file to match a reference node and track the errors it
triggers.
We also no longer use gtester.

rendernodeparser: Skip root node when it's a container

When printing, behave the same way as when parsing:
Magically skip a container node if there is one - just like the
parser magically creates a container node to hold all the nodes
it parses.

rendernodeparser: Allow single values instead of 4

This allows writing:
  colors: red;
instead of
  colors: red red red red;
to draw a red border.

rendernodeparser: Print properties alphabetically

Because now we have a rule.

Also update some tests to the new order.

glrenderer: Handle NULL debug messages

gtk-demo: Fix menubutton usage in listbox demo

Menu buttons are no longer buttons, so they
can't have children, and don't have a relief
property.

inspector: Don't apply button api to menu buttons

gl renderer: Pass correct matrix category when rendering offscreen

We can't just pretend we have an identity matrix when we are actually
scaling. This fixes the node editor sometimes not drawing things when
rendering to a texture. We were mistakenly discaring render nodes
because the bounds transformation was wrong.

gl renderer: Delete render_texture framebuffer

gl renderer: A GdkGLTexture's context might be NULL

gdk_gl_texture_download ought to still work, but we can't make a NULL
context current obviously.

gl renderer: Really fix gl debugging code

Properly label an object and make sure we have the right gl context even
after a gdk_gl_context_end_frame call.

builder-tool: Remove debug spew

x11: Remove to unused settings

We no longer have settings for gtk-button-images
or gtk-menu-images.

lock button: Don't allow subclassing

link button: Don't allow subclassing

Fix a compiler warning

popover menu: Make not subclassable

popover: Cosmetics

Use the same amount of padding as everywhere else.

volume button: Don't allow subclassing

If you want to subclass, there's GtkScaleButton.

settings: Drop the priv pointer

application window: Drop the priv pointer

sizegroup: Drop the priv pointer

settings: Make not subclassable

Part of being more explicit about what
we allow to be subclassed and what not.

Forgotten file

shortcuts window: Make not subclassable

Part of being more explicit about what
we allow to be subclassed and what not.

css provider: Make not subclassable

Part of being more explicit about what
we allow to be subclassed and what not.

size group: Make not subclassable

Part of being more explicit about what
we allow to be subclassed and what not.

password entry: Make not subclassable

Part of being more explicit about what
we allow to be subclassed and what not.

Padding review

Ensure that the class structs of all subclassable
types have sufficient padding (standardizing on 8
slots, here).

GtkBox
GtkButton
GtkDrawingArea
GtkFixed
GtkFrame

Revert "wayland: Make popups work more than once"

This reverts commit e74c655016982272b72dff318b11602b0d5b099a.

message dialog: Make not subclassable

Part of being more explicit about what
we allow to be subclassed and what not.

Padding review

Ensure that the class structs of all subclassable
types have sufficient padding (standardizing on 8
slots, here).

GtkApplication
GtkWidget
GtkContainer
GtkWindow
GtkDialog
GtkApplicationWindow
GtkToolItem
GtkBin

Cosmetic changes

Use new-style padding for GdkContentProviderClass.

Trivial: Remove a few +'s

Merge branch 'frame-pixel-counter' into 'master'

profiling: Add a counter for pixels drawn per frame

See merge request GNOME/gtk!855

wayland: Make popups work more than once

The change to keep some server resources around
until destroy was causing us to not recreate
the right things when a surface is hidden and
then shown again. Make sure to recreate everything.

widget-factory: menu buttons can't have children

GtkMenuButton is no longer a container, so we
can't pack children in it.

Merge branch 'wayland-surface-unmap' into 'master'

wayland: Keep some resources until destroy

Closes #1485

See merge request GNOME/gtk!857

wayland: Keep some resources until destroy

The Wayland backend was dropping _all_ serverside
resources on hide, which is too early e.g. for
GtkGLArea which wants to use egl resources to
unload textures on unrealize.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1485

Merge branch 'button-clicked' into 'master'

Drop gtk_button_clicked

See merge request GNOME/gtk!856

Drop gtk_button_clicked

This is not an api we want to propagate anymore.
If you need to, you can still emit the "clicked"
action signal on a button using g_signal_emit_by_name.

demos: Stop using gtk_button_clicked

gtk: Stop using gtk_button_clicked

This function is going away.

profiling: Add a counter for pixels drawn per frame

This number clearly shows the recently discovered
"full redraws" problem.